BetterGenshinImpact/GameTask/AutoFight/AutoFightParam.cs (1)

82-82: 为新开关补充注释与命名对齐（可选）

建议为 ExpKazuhaPickup 增补 XML 注释，说明与万叶/琴“聚集拾取”的关系；同时确认 UI 绑定字段命名与此属性完全一致，避免歧义。

BetterGenshinImpact/GameTask/AutoFight/Assets/AutoFightAssets.cs (2)

47-52: ExperienceRa 单实例易被覆盖；_gameScreenSize 初始化时序有隐患（可改进）

• InitializeRecognitionObject 会被多次调用生成不同经验模板，但类字段 ExperienceRa 每次都会被覆盖，后续调试显示/复用易混淆。建议移除该字段或改为 Dictionary<int, RecognitionObject> 缓存。
• _gameScreenSize 直接取 SystemControl.GetGameScreenRect(TaskContext.Instance().GameHandle) 依赖初始化时序。更稳妥做法是使用注入的 systemInfo.GameScreenSize，保持与其余素材缩放逻辑一致。

---

274-298: InitializeRecognitionObject：建议使用 systemInfo 宽度、避免写全局字段并默认关闭绘制

当前实现功能可用，但可小幅优化以降低耦合和开销：

• 使用 this.systemInfo.GameScreenSize.Width 判定阈值，避免依赖 _gameScreenSize。
• 不写入类字段 ExperienceRa，直接返回新对象；或按经验值缓存。
• DrawOnWindow 默认应为 false（仅调试时开启），减少遮罩层刷新。

可参考如下最小变更：

-    public RecognitionObject InitializeRecognitionObject(int experience)
+    public RecognitionObject InitializeRecognitionObject(int experience)
     {
-        var threshold = 0.9;
-        
-        if (_gameScreenSize.Width > 2560)
-        {
-            threshold =0.6;
-        }
-        else if (_gameScreenSize.Width > 1920)
-        {
-            threshold =0.6;
-        }
-        
-        ExperienceRa = new RecognitionObject
+        var width = this.systemInfo.GameScreenSize.Width;
+        var threshold = width > 1920 ? 0.6 : 0.9;
+
+        var ro = new RecognitionObject
         {
             Name = experience.ToString(),
             RecognitionType = RecognitionTypes.TemplateMatch,
-            TemplateImageMat = GameTaskManager.LoadAssetImage("AutoFight", "experience_" + experience + ".png"),
+            TemplateImageMat = GameTaskManager.LoadAssetImage("AutoFight", $"experience_{experience}.png", this.systemInfo),
             RegionOfInterest = new Rect((int)(CaptureRect.Width*0.145),(int)(CaptureRect.Height*0.5), (int)(CaptureRect.Width*0.02), (int)(CaptureRect.Height*0.22)),
             UseMask = true,
             Threshold = threshold,
-            DrawOnWindow = true,
+            DrawOnWindow = false,
         }.InitTemplate();
-        return  ExperienceRa;
+        return ro;
     }  

BetterGenshinImpact/GameTask/AutoFight/AutoFightTask.cs (1)

27-35: 重复 using 可清理（可选）

BetterGenshinImpact.GameTask.AutoFight.Assets 被重复导入，建议删除重复 using，减少噪音。

BetterGenshinImpact/View/Pages/View/ScriptGroupConfigView.xaml (2)

1428-1476: “聚集材料动作”新增列与开关布局可读性改进建议

• Toggle 与说明 TextBlock 同格叠放（同列同 RowSpan）易出现遮挡/对齐偏差。建议在每个开关所在单元格内部使用垂直 StackPanel（上 Toggle、下 8pt 标签），更稳健。
• IsEnabled 绑定一般应为 OneWay，避免 UI 状态回写 VM。当前对 KazuhaPickupEnabled 的 IsEnabled 采用 TwoWay 没必要。

建议修改如下：

- IsEnabled="{Binding PathingConfig.AutoFightConfig.KazuhaPickupEnabled, Mode=TwoWay}"
+ IsEnabled="{Binding PathingConfig.AutoFightConfig.KazuhaPickupEnabled}"  <!-- OneWay 默认 -->

同时可将每个 ToggleSwitch 与其说明 TextBlock 包装进一个 StackPanel 后再放入该列，避免同格叠放导致的可视/命中问题。

---

1442-1443: 说明文字统一与标点修正

两处页面对“基于经验/琴二次拾取”的描述略有出入，且混用英文逗号。建议统一文案并使用中文标点，便于本地化一致性。

- Text="(基于经验：通过精英经验值判断是否聚集 / 琴二次拾取：琴首次拾取空,再次拾取)"
+ Text="(基于经验：通过精英经验值判断是否聚集 / 琴二次拾取：首次拾取为空，再次尝试拾取)"

BetterGenshinImpact/View/Pages/TaskSettingsPage.xaml (1)

1024-1094: “聚集材料动作”四列布局的小优化（绑定方向与布局）

• 两个说明 TextBlock 与各自 Toggle 同格叠放，容易在不同 DPI/缩放下产生遮挡。建议在每个 Toggle 所在单元格内包一层垂直 StackPanel，Toggle 在上、8pt 说明在下。
• IsEnabled 对 KazuhaPickupEnabled 应用 TwoWay 没必要，改 OneWay 更稳。

建议修改：

- IsEnabled="{Binding Config.AutoFightConfig.KazuhaPickupEnabled, Mode=TwoWay}"
+ IsEnabled="{Binding Config.AutoFightConfig.KazuhaPickupEnabled}"

另外，建议将本页与 ScriptGroupConfigView.xaml 的说明文字保持一致，并统一中文标点：

- Text="(基于经验：检测精英经验值判断聚集 / 琴二次拾取：首次拾取空,再次拾取)"
+ Text="(基于经验：通过精英经验值判断是否聚集 / 琴二次拾取：首次拾取为空，再次尝试拾取)"

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

BetterGenshinImpact/GameTask/AutoFight/AutoFightTask.cs (3)

58-58: 跨线程访问 _isExperiencePickup 缺少内存可见性保证（竞态条件）

该标志在 FindExp 的后台任务中写入（第 777、797、806 行），在主流程中读取（第 496、507、513 行）。当前为普通静态字段，存在内存可见性风险，线程可能读取到旧值。

建议将其声明为 volatile 以确保跨线程内存可见性。

-    private static bool _isExperiencePickup = false;
+    private static volatile bool _isExperiencePickup = false;

---

318-322: FindExp 任务生命周期未管理，清理时机错误

FindExp 立即返回 CompletedTask，真正的检测在 Task.Run 内异步进行。同时 ClearAll 在 FindExp 方法的外层 finally 中被过早调用（第 839-842 行），无法保证在任务结束后才清理绘制内容。

建议让 FindExp 返回内部 Task，并将清理移入该任务的 finally 块中。调用端可根据需要忽略或等待返回的 Task。

参见第 768-845 行 FindExp 方法的详细修改建议。

---

768-845: FindExp 方法存在多个未解决的关键问题

该方法存在以下问题（与先前评审意见一致）：

1. 生命周期管理错误：返回 Task.CompletedTask 而实际检测在 Task.Run 内部异步执行，导致调用方无法追踪任务状态
2. 阻塞调用：第 812 行使用 .Result 阻塞等待，应改用 await
3. 清理时机错误：第 839-842 行的 ClearAll 在外层 finally 中执行，会在后台任务完成前被调用
4. 缺少边界检查：第 805 行 isExist.X - 147 可能导致负数或越界访问
5. 颜色匹配过于严格：第 806 行使用精确 RGB 匹配，不同分辨率/抗锯齿下容易误判

建议的修复方案（来自先前评审）：

-    private static Task FindExp(CancellationToken cts2)
+    private static Task FindExp(CancellationToken token)
     {
         var autoFightAssets = AutoFightAssets.Instance;
-        try  
-        {
-            Task.Run(() =>
+        return Task.Run(async () =>
+        {
+            try
             {
                 _isExperiencePickup = false;
-                var expLogo = false;
                 
                 var experienceRas = new[]
                 {
                    autoFightAssets.InitializeRecognitionObject(60), 
                    autoFightAssets.InitializeRecognitionObject(58), 
                    autoFightAssets.InitializeRecognitionObject(57),
                 };
                 
-                while (!(_isExperiencePickup || !FightStatusFlag) && !cts2.IsCancellationRequested)
+                while (!_isExperiencePickup && FightStatusFlag && !token.IsCancellationRequested)
                 {
                     try
                     {
-                        cts2.ThrowIfCancellationRequested();
-
-                        var result = NewRetry.WaitForAction(() =>
+                        token.ThrowIfCancellationRequested();
+                        await NewRetry.WaitForAction(() =>
                         {
                             using (var ra = CaptureToRectArea())
                             {
-                                _isExperiencePickup = experienceRas.Any(experienceRa => 
+                                foreach (var experienceRa in experienceRas)
                                 {
                                     var isExist = ra.Find(experienceRa);
                                     if (!isExist.IsExist())
                                     {
-                                        return false;
+                                        continue;
                                     }
-                
-                                    var pixelValue1 = ra.SrcMat.At<Vec3b>(isExist.Y, isExist.X - 147);
-                                    expLogo = pixelValue1[0] == 253 && pixelValue1[1] == 247 && pixelValue1[2] == 172;
-
-                                    return expLogo;
-                                });
+                                    
+                                    var sampleX = isExist.X - 147;
+                                    var sampleY = isExist.Y;
+                                    
+                                    // 边界检查
+                                    if (sampleX < 0 || sampleY < 0 || 
+                                        sampleX >= ra.SrcMat.Cols || sampleY >= ra.SrcMat.Rows)
+                                    {
+                                        continue;
+                                    }
+                                    
+                                    var p = ra.SrcMat.At<Vec3b>(sampleY, sampleX);
+                                    // 颜色容差匹配（±3）提升鲁棒性
+                                    bool colorMatch = Math.Abs(p[0] - 253) <= 3 && 
+                                                     Math.Abs(p[1] - 247) <= 3 && 
+                                                     Math.Abs(p[2] - 172) <= 3;
+                                    
+                                    if (colorMatch)
+                                    {
+                                        _isExperiencePickup = true;
+                                        break;
+                                    }
+                                }
                             }
                             return _isExperiencePickup;
-                        }, cts2, 1, 100).Result;
+                        }, token, 1, 100);
+                        
+                        if (_isExperiencePickup)
+                        {
+                            Logger.LogInformation("基于怪物经验判断：识别到 {text1} 经验值，{text2} 聚集拾取", "精英", "启用");
+                        }
                     }
-                    catch (OperationCanceledException ex)
-                    {
-                        Console.WriteLine($"检测经验发生异常: {ex.Message}");
-                    }
-                    catch (Exception ex)
-                    {
-                        // Console.WriteLine($"检测怪物经验发生异常: {ex.Message}");
-                    }
-                    
-                    if (_isExperiencePickup) Logger.LogInformation("基于怪物经验判断：识别到 {text1} 经验值，{text2} 聚集拾取","精英","启用" );
-
+                    catch (OperationCanceledException) { /* 正常取消，忽略 */ }
+                    catch { /* 检测异常，继续循环 */ }
                 }
-                
-                cts2.ThrowIfCancellationRequested();
-                
-            }, cts2); 
-        }
-        catch (OperationCanceledException ex)
-        {
-            Console.WriteLine($"检测经验发生异常: {ex.Message}");
-        }
-        catch (Exception ex)
-        {
-            Console.WriteLine($"检测怪物经验发生异常: {ex.Message}");
-        }
-        finally
-        {
-            VisionContext.Instance().DrawContent.ClearAll();
-        }
-        
-        return Task.CompletedTask;
+            }
+            finally
+            {
+                VisionContext.Instance().DrawContent.ClearAll();
+            }
+        }, token);
     }

BetterGenshinImpact/GameTask/AutoFight/AutoFightTask.cs (3)

26-26: 删除重复的 using 声明

第 26 行和第 34 行都导入了 BetterGenshinImpact.GameTask.AutoFight.Assets，存在重复。

 using BetterGenshinImpact.GameTask.AutoFight.Assets;
 using BetterGenshinImpact.View.Drawable;
 using BetterGenshinImpact.Core.Recognition.OpenCv;
 using BetterGenshinImpact.GameTask.Common.BgiVision;
 using Vanara.PInvoke;
 using BetterGenshinImpact.GameTask.AutoPathing.Handler;
 using BetterGenshinImpact.GameTask.AutoPick.Assets;
 using BetterGenshinImpact.GameTask.AutoPathing.Model;
-using BetterGenshinImpact.GameTask.AutoFight.Assets;

Also applies to: 34-34

---

496-500: 优化延迟条件逻辑

当 _isExperiencePickup 为 false 时添加延迟等待经验显示的逻辑是合理的，但建议明确日志输出的含义。当前代码在所有满足条件的情况下都会延迟 1 秒，可能影响已检测到经验的情况。

建议确认此延迟仅在确实需要等待经验值显示时执行。

---

507-510: 复杂的布尔条件可考虑添加注释

第 507 行的条件逻辑正确，但较为复杂。建议添加注释说明何时跳过拾取：

• 战斗人次低于阈值，且（未启用经验值判断 或 未检测到经验值）时跳过。

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

BetterGenshinImpact/GameTask/AutoFight/AutoFightTask.cs (2)

318-322: FindExp 调用未管理生命周期；请改为返回内部 Task 并在调用处显式持有/忽略

当前 FindExp(cts2.Token) 内部再 Task.Run 且方法本身返回 Task.CompletedTask，调用方无法管理其生命周期；外层清理也可能过早执行。建议让 FindExp 直接返回内部任务，并在此处显式持有（或用 _ = 明确忽略）。

-                if (_taskParam.ExpKazuhaPickup && findExpAvatar) FindExp(cts2.Token);
+                if (_taskParam.ExpKazuhaPickup && findExpAvatar)
+                {
+                    // 显式启动并忽略返回的 Task，表明有意异步执行且不等待
+                    _ = FindExp(cts2.Token);
+                }

---

768-845: FindExp 存在多处并发与正确性隐患：返回值、清理时机、.Result 阻塞、像素越界与颜色容忍度

• 方法返回 CompletedTask，而真实工作在内部 Task，调用方无法管理其生命周期；外层 finally 的 ClearAll() 可能比内部任务先执行，导致闪烁/残留。
• 使用 NewRetry.WaitForAction(...).Result 在线程池线程上同步阻塞，易造成线程饥饿与不可控延迟。
• 采样点 isExist.X - 147 未做边界检查，可能越界访问 SrcMat。
• 使用严格 RGB 等值判断，易受分辨率/抗锯齿影响误判。
• 循环条件读取 FightStatusFlag 为普通属性（非 volatile/Interlocked），跨线程可见性无保证。

建议重构如下：

• 让 FindExp 返回内部 Task，并将 ClearAll() 放入其 finally。
• 用 await 调用 WaitForAction，避免 .Result。
• 对采样坐标做边界检查。
• 颜色判断加入微小容忍度（如 ±3）。
• 为 FightStatusFlag 增加内存可见性保证（见下方附加 diff）。

-    private static Task FindExp(CancellationToken cts2)
-    {
-        var autoFightAssets = AutoFightAssets.Instance;
-
-        try  
-        {
-            Task.Run(() =>
-            {
-                _isExperiencePickup = false;
-                var expLogo = false;
-                
-                var experienceRas = new[]
-                {
-                   autoFightAssets.InitializeRecognitionObject(60), 
-                   autoFightAssets.InitializeRecognitionObject(58), 
-                   autoFightAssets.InitializeRecognitionObject(57),
-                };
-                
-                while (!(_isExperiencePickup || !FightStatusFlag) && !cts2.IsCancellationRequested)
-                {
-                    try
-                    {
-                        cts2.ThrowIfCancellationRequested();
-
-                        var result = NewRetry.WaitForAction(() =>
-                        {
-                            using (var ra = CaptureToRectArea())
-                            {
-                                _isExperiencePickup = experienceRas.Any(experienceRa => 
-                                {
-                                    var isExist = ra.Find(experienceRa);
-                                    if (!isExist.IsExist())
-                                    {
-                                        return false;
-                                    }
-                
-                                    var pixelValue1 = ra.SrcMat.At<Vec3b>(isExist.Y, isExist.X - 147); //经验值图标，在2K以上时匹配度0.6，这个经验值颜色尤为重要
-                                    expLogo = pixelValue1[0] == 253 && pixelValue1[1] == 247 && pixelValue1[2] == 172;
-
-                                    return expLogo;
-                                });
-                            }
-                            return _isExperiencePickup;
-                        }, cts2, 1, 100).Result;
-                    }
-                    catch (OperationCanceledException ex)
-                    {
-                        Console.WriteLine($"检测经验发生异常: {ex.Message}");
-                    }
-                    catch (Exception ex)
-                    {
-                        // Console.WriteLine($"检测怪物经验发生异常: {ex.Message}");
-                    }
-                    
-                    if (_isExperiencePickup) Logger.LogInformation("基于怪物经验判断：识别到 {text1} 经验值，{text2} 聚集拾取","精英","启用" );
-
-                }
-                
-                cts2.ThrowIfCancellationRequested();
-                
-            }, cts2); 
-        }
-        catch (OperationCanceledException ex)
-        {
-            Console.WriteLine($"检测经验发生异常: {ex.Message}");
-        }
-        catch (Exception ex)
-        {
-            Console.WriteLine($"检测怪物经验发生异常: {ex.Message}");
-        }
-        finally
-        {
-            VisionContext.Instance().DrawContent.ClearAll();
-        }
-        
-        return Task.CompletedTask;
-    }
+    private static Task FindExp(CancellationToken token)
+    {
+        var assets = AutoFightAssets.Instance;
+        return Task.Run(async () =>
+        {
+            try
+            {
+                _isExperiencePickup = false;
+                var experienceRas = new[]
+                {
+                    assets.InitializeRecognitionObject(60),
+                    assets.InitializeRecognitionObject(58),
+                    assets.InitializeRecognitionObject(57),
+                };
+
+                while (!_isExperiencePickup && FightStatusFlag && !token.IsCancellationRequested)
+                {
+                    try
+                    {
+                        token.ThrowIfCancellationRequested();
+                        var detected = await NewRetry.WaitForAction(() =>
+                        {
+                            using var ra = CaptureToRectArea();
+                            foreach (var ro in experienceRas)
+                            {
+                                var match = ra.Find(ro);
+                                if (!match.IsExist()) continue;
+                                var sx = match.X - 147;
+                                var sy = match.Y;
+                                if (sx < 0 || sy < 0 || sx >= ra.SrcMat.Cols || sy >= ra.SrcMat.Rows) continue;
+                                var p = ra.SrcMat.At<Vec3b>(sy, sx);
+                                const int tol = 3;
+                                bool near =
+                                    Math.Abs(p[0] - 253) <= tol &&
+                                    Math.Abs(p[1] - 247) <= tol &&
+                                    Math.Abs(p[2] - 172) <= tol;
+                                if (near) return (_isExperiencePickup = true);
+                            }
+                            return _isExperiencePickup;
+                        }, token, retryTimes: 1, delayMs: 100);
+
+                        if (detected)
+                        {
+                            Logger.LogInformation("基于怪物经验判断：识别到 {text1} 经验值，{text2} 聚集拾取", "精英", "启用");
+                        }
+                    }
+                    catch (OperationCanceledException) { /* ignore */ }
+                    catch { /* ignore */ }
+                }
+            }
+            finally
+            {
+                VisionContext.Instance().DrawContent.ClearAll();
+            }
+        }, token);
+    }

附：为 FightStatusFlag 增加可见性保证（在属性定义处修改）。

-public static bool FightStatusFlag { get; set; } = false;
+private static bool _fightStatusFlag;
+public static bool FightStatusFlag
+{
+    get => Volatile.Read(ref _fightStatusFlag);
+    set => Volatile.Write(ref _fightStatusFlag, value);
+}

BetterGenshinImpact/GameTask/AutoFight/AutoFightTask.cs (2)

27-35: 移除重复 using 以消除冗余

using BetterGenshinImpact.GameTask.AutoFight.Assets; 出现两次（行 27 与行 34）。保留一次即可。

-using BetterGenshinImpact.GameTask.AutoFight.Assets;

---

496-501: 等待经验显示时使用局部取消令牌，避免外部取消不一致

此处建议用 cts2.Token 保持与本次战斗生命周期一致；并建议在日志后快速返回而非固定 1s 等待（可由 WaitForAction 驱动）。

-                    await Delay(1000, ct);
+                    await Delay(1000, cts2.Token);

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro